Update changelog for 1.6.0+ds-2 release

Build doc in build-indep

Update changelog for 1.6.0+ds-1 release

Bump policy version (no changes)

Maintainer set to team and add myself as uploader

Update Vcs-Git Urls.

Use gitlab-ci instead of travis

Update changelog for 1.6.0-1 release

Update copyright

Port to dh 12

Closes: #912089

Add patches to use system libraries

Rebase patches

Add watch file

Fix debian/source/format

Merge branch 'upstream'

Cleanup upstream sources

Revert "Add a debian .travis.yml"

This reverts commit fb44d9868f16e5dd4f5d4187e645eb4e304a81ce.

Merge tag 'gpsbabel_1_6_0' into upstream

Merge pull request #346 from tsteven4/r160

Roll version number for release 1.6.0.

Roll version number for release 1.6.0.

Merge pull request #345 from gpsbabel/gpsbabel-patch-2

Remove outdated multiline comment in gdb.cc

Merge pull request #340 from tsteven4/qt5123

Roll Qt to 5.12.3 and 5.9.8

Remove outdated multiline comment in gdb.cc

hook GUI up to sort filter. (#342)

because the sort filter now works on waypoints
and/or routes and/or tracks the dialog is under
the Miscellaneous menu.

pull minizip from configure flow. (#339)

while it was planned to use minizip those
plans were never completed and it was unused.

zlib/contrib/minizip appears to be unmaintained.
some issues e.g. #338.

Roll Qt to 5.12.3 and 5.9.8

Merge pull request #336 from gpsbabel/datefix

Fix date in Windows rc copyright.

Fix date in Windows rc copyright.

Fix order of arguments to new upgrade check.

Check in the regenerated setup.iss

Turn off debugging mode in upgrade check.

Fix GUI upgrade check to handle upgrade from x-betaFOO to x.

Bump version number for (hopefully short-lived) beta.

Update dependencies in Makefile.in for release.

split csv lines as QStrings and return a list of all fields. (#332)

* introdcue csv_linesplit

Which is like csv_lineparse, except it gives you a list
of all the values, and it processing the line as a QString.

* introduce gpsbabel::TextStream, use it

in unicsv reader, ozi.

* convert unicsv writer to textstream.

* if a codec is not found list available.

* switch unicsv to CET_CHARSET_UTF8 to avoid

undesired fs conversions.  All conversions are
handled by the codec used by qtextstream.

* output boms with non utf8 unicode codecs.

* use rfc4180 dequote method with unicsv.

* add test for csv quoting w RFC4180.

add documentation for some options. (#334)

* add documentation for some options.

garmin_gpi: languagecode
gpx: elevprec

* doc wording tweak.

* delete obsolete unused lowranceusr option,

and document new lowranceusr options.

* tweak doc.

update continous release version for macos. (#331)

update Qt 5.12 builds to 5.12.2. (#329)

* update Qt 5.12 builds to 5.12.2.

* update coverage reporter, kill java8 in docker, add bear.

retire queues. (#328)

 Introduce a WaypointList class to define the interface to our lists of waypoints. (#326)

* Introduce a WaypointList class to define the interface to our lists of waypoints.

WaypointList is backed QList, eliminating the use of legacy queues for waypoint lists.

* try to appease xcode wrt std::abs.

* clean up inheritence in WaypointList, RouteList.

also quiet clazy with igc and gpsbabel_optional::optional.
also improve includes in defs.h.

* get rid of unnecessary reinterpret_cast.

and correct pointer constness lost by auto.

replace queues in mkshort. (#327)

Also, hide the contents of mkshort_handle_imp.

fiddle with nmea waypoint status tracking. (#325)

Traditionally nmea looks at the Waypoint data member Q
to decide if the Waypoint has been added on to a global list, and
if it has not it deletes the Waypoint so we don't leak memory.

This is problematic when we change the waypoint lists so queues
aren't used.

To overcome this difficulty nmea now uses a derived class of
Waypoint that tracks if the Waypoint has been added or not.

fix stackfilter issues. (#324)

1. correct defect in the new RouteList::swap method.
2. correct an old bug in stackfilter that cleaned up
waypts on exit, but not routes or tracks.
3. enhance stackfilter test to test routes and tracks as well
as waypts.

replace random queues with QLists. (#323)

Merge pull request #296 from tsteven4/routelist

Introduce a RouteList class.

convert ozi io to QTextStream (#315)

* convert ozi io to QTextStream.

and add an option to set the codec.
default the codec to windows-1252, which matches historic
usage but not recent behavior.

fix a memory leak, csv_lineparse needs to die.

fix a mistranslation with QString::arg.

* enhance ozi test for routes and tracks.

* eliminate csv_lineparse in ozi.

* eliminate obsolete commented code in ozi.

* update encoding comments for ozi.

* add doc for ozi codec option.

regenerate translation files including command line strings. (#319)

* spelling fixes in mainwindow.cc

* regenerate compiled translation files.

add a process to capture the translated command line strings, &
update the corresponding translation source and compiled files.

remove gui context from gpsbabel_it.ts.

correct errors in mainwindow setting up command line translations.

deploy compiled command line translations.

* correct permissions on new pro file.

* delete obsolete and empty translation files.

add CMakeLists.txt file for GUI. (#322)

Merge pull request #298 from gpsbabel/skytraq

Essentially revert the skytraq part of PR#286.

Merge pull request #321 from gpsbabel/gpsbabel-patch-3

Drop unused get_crc32_s

Merge pull request #320 from gpsbabel/gpsbabel-patch-2

Remove dead code: get_crc32_s

Merge branch 'master' into skytraq

Drop unused get_crc32_s

Remove dead code: get_crc32_s

clean up gui main and fix a png. (#313)

* clean up gui main and fix a png.

- fix libpng warning: iCCP: known incorrect sRGB profile
  issue with runit.png with pngcrush.

- fix includes for gui/main.cc, including removal of
  CoreFoundation.h
- use QApplication static methods instead of going directly
  to the underlying QCoreApplication static methods.

 - no path setting in GUI, instead pass the full path to gpsbabel when using QProcess::start.

in formatload.cc xlt() use qt methods to get a char* instead of
resorting to stdlib.  At this point Qt assumes all translate strings
are utf8.  However, the formatload xlt intent doesn't work because
1) MainWindow::LoadLanguage hasn't been called when
   MainWindow::loadFormats is called.
2) the translations don't exist for the format bits from gpsbabel
   that fromatload tries to translate.  I think this was the intent
   of the gpsbabel_*.ts files which are very out of date and no
   longer distributed.

spelling fixes in mainwindow.cc (#318)

normalize connections in GUI (#316)

as suggested by clazy.

Merge pull request #312 from tsteven4/tidygui3

clang-tidy fixes for the gui

clang-tidy readability-else-after-return on gui.

and reformat fixes with astyle.

manual fix suggested by clang-tidy modernize-use-using.

clang-tidy readability-named-parameter on gui.

clang-tidy hicpp-use-override on gui.

clang-tidy readability-simplify-boolean-expr on gui.

clang-tidy readability-braces-around-statements on gui.

clang-tidy readability-static-accessed-through-instance on gui.

clang-tidy readability-implicit-bool-conversion on gui.

clang-tidy modernize-use-nullptr on gui.

clang-tidy readability-delete-null-pointer on gui.

clang-tidy readability-container-size-empty on gui.

clang-tidy readability-redundant-member-init on gui.

clang-tidy performance-unnecessary-value-param on gui.

clang-tidy hicpp-use-equals-default on gui.

reformat gui with astyle. (#311)

incorporate review changes.

Refactor application packaging for linux/macos. (#310)

* Refactor application packaging for linux/macos.

Packaging of the GUI for linux/macos can now be done
either by building the make target "package" created by
"qmake app.pro", or by executing a bash script "package_app".
Packaging with the legacy configure flow will build the
package target.
Travis CI scripts will build the package target.

The obsolete script mac-localize is deleted.  This
functionality is included in package_app.

The obsolete script makelinuxdist.sh is deleted.  We have not
distributed a linux package in some time.  Furthermore this
scirpt hadn't worked in some time.  package_app does not
create a standalone linux app, however it packages all the
bits from GPSBabel in GPSBabelFE.  This makes creating a
Docker app, which includes the Qt installation, for GPSBabel easy.

Merge relevant Qt .qm files into one qt_xx.qm file for each language
as recommended by Qt.
Deploy these meged .qm files with linux/macos builds.
Since Qt5 we only deployed the meta catalog, but not it's
dependencies. This should have led to the dependencies not
being available without Qt being installed.

gpsbabel.qm and gpsbabelfe.qm are no longer included in the packaged
apps for windows/linux/macos, as it should be.

Merge pull request #294 from madam7/master

skytraq updates: prepare gps week rollover, and an older bugfix

Merge branch 'master' into routelist

including manual resolution of conflict in main.cc.

Object cleanup (#307)

* elminiate some early program exits.

exits without cleanup make it harder to find memory leaks.

fix a few leaks.

enhance random format to support reading real time position data.

correct some sign related warnings.

* use invariant generator for random format.

* make compilers happy with random.

clang doesn't like:
  random.cc:61:42: note: read of non-constexpr variable 'generator' is not allowed in a constant expression
    constexpr double scalefactor = 1.0 / (*generator).max();

msvc doesn't like:
  warning C4100: 'fname': unreferenced formal parameter

* catch main.cc up with master manually.

Change diff and terrain back to doubles in geo

Not part of official geo format that's widely used; apparently only used by rickrich's geo tools.

Qt 5.12.1 and valgrind on Linux (#306)

set an environmental variable when running valgrind to disable QRegularExpressions use of JIT which causes problems for valgrind. The release build of Qt 5.12.1 uses JIT by default. The debug build does not.

warn when running older version of valgrind that don't handle RDRAND instructions used by newer versions of Qt such as 5.12.1.

support multiple docker images in travis.

create a travis docker image with Qt 5.12.1.

do parallel builds for travis macos. (#305)

Set Qt version floor at 5.9 (#304)

* move Qt floor to 5.9.

* update coverage build for Qt archive naming.

* update all travis builds to xenial.

trusty is EOL 4/2019.

* avoid 5.12.1 on linux CI.

* Revert "avoid 5.12.1 on linux CI."

This reverts commit b86212123cda9d7c57f212f3baf0f05bc3121ee5.

* avoid 5.12.1 on linux CI.

Parallel test (#303)

* run vtesto jobs in parallel.

with the configure flow and "make check-vtesto".
This will be utilized on travis.

* add missing gitignore for new valgrind logs.

fix pcx test to check output it generates! (#302)

update appveyor for Qt 5.12.1 (#301)

* update appveyor builds.

archive msvc2017,nmake, Qt 5.12,x86.
switch all msvc2017 builds to Qt 5.12.

* work around qmake 5.12.1 bug.

reorder builds so release candidate is done first.

archive macos with Qt 5.12.1 (#300)

instead of 5.9.7.

Merge branch 'master' into skytraq

Essentially revert the skytraq part of PR#286.

Merge pull request #293 from tsteven4/trackf

modernization of track filter

Merge pull request #271 from gromit1811/fit-trkseg

Start new track segments on Garmin Fit timer events (start/stop/...)

Relocate xcsv format functions. (#297)

Many funtions for the xcsv format were in csv_util.cc
instead of xcsv.cc.  This commit moves the xcsv format
specific functions to xcsv.cc, and leaves the possibly
shared utility functions in csv_util.cc.

xcsv.h is added to communicate shared xcsv style handling
with vecs.cc.

Added support for USR version 5 and 6 and corrected USR version 4 support. (#268)

* Minor changes to address GCC 7.3.0 warnings generated by flag -Wunused-but-set-variable.

Moved declaraions/initialization of rp_backup and era inside #ifdef _NL_CURRENT conditionals.

* Added { ... } to for at the end of zip64local_SearchCentralDir() so that if-break statements inside for loop.

* Removed declaration and commented out references to startlat, startlon, and starttime in fit_parse_data() due to GCC 7.3.0 warning from
flag -Wunused-but-set-variable.

* Added initial support for Lowrance USR 5 and 6 formats.

* Significant changes to handle multiple USR4 versions and add USR5 and USR6 support.

* Remove extraneous debug NL when no debug specified.

* Updated USR4 reference file because of increased accuracy in converting lat/log values.

* Documentation on some ICON mappings.

* Added USR5 and USR6 to validation checks.

* Added reference files for USR5 and USR6 formats.

* Cleaned up some debug output.

* Continued work to consolidate all Lowrance USR processing into a single file.

* Eliminated lowranceusr4.cc and restuctured testing to include USR4, USR5, and USR6 formats.

* Changes associated with adding support for USR 5 and USR 6.

* Eliminated lowranceusr4.cc from build.

* Changed default Output USR Version from 4 to 2 for backward compatibilty.

* Removed USR 4 section from documentation.

* Initial update of XMLDOC for updated data formats.

* iMinor updates to parse_routes.  Added correct v4 and v5 GPX reference files.

* Documentation updates to reflect new capabilities.

* Added Route element information to document and minor code changes to match.

* Doc refinement with rewrite.

* Corrections to USR 4 output.

* Update to correct v4 gpx file.

* Checkin correct v5 gpx file.

* Removed errant NL when no debug specified.

* Resolved conflicts

* Missed this update.

* Revert "Minor changes to address GCC 7.3.0 warnings generated by flag -Wunused-but-set-variable."
at the request of tsteven4 as they have nothing to do with the expanded support.

This reverts commit fb93a19b740edae4dde3b6dcd0546f344122f70d.

* Revert "Added { ... } to for at the end of zip64local_SearchCentralDir() so that if-break statements inside for loop."
at the request of tsteven4 as they have nothing to do with the expanded support.

This reverts commit 189bbe6ea664829412c70079e9ddb9555062a12f.

* Revert "Removed declaration and commented out references to startlat, startlon, and starttime in fit_parse_data() due to GCC 7.3.0 warning from"
at the request of tsteven4 as they have nothing to do with the expanded support.

This reverts commit cc618336888c5cdb8adef87f72026a8c24eff7e8.

* Updated to remove lowranceusr4.cc per suggestion from tsteven4 to support cmake/XCLion.

* Per suggestions from tsteven4
    - deleted define SECSTORADIANS as it was not used
    - ran astylerc

* Replaced use of NULL with nullptr per tsteven4 suggestion.

* Added URLs for reference information.

* Adopted tsteven4 recommendation on eliminating compiler warning.

* Corrections to Waypoint and Trail documentation for USR 4, 5, and 6 formats.

* Address Codacy flagged issues.

* Address Codacy flagged issues.

* Address additional Codact flaged issues.

* Changed DEGREESTORADIANS define from hardcoded value to use (M_PI/180.0)

* Removed Copyright protected material and replaced with authored comments.

* Corrected altitude/depth retension coverting USR4/5/6 to USR2/3

* Added mapping of USR4/5/6 ICON numbers to text to support output to USR 2/3 format.

* Corrected generation of USR4 data file.

* Check for NULL format specific when converting USR 2/3 (colorless) to USR4 (color coded) ICONs.

* Added cases where USR 2/3 embeds color in shortname.

* Corrections to ICON translation from USR v2/3 to USR 4.

* Cleanup USR 2/3 parsing debug.

* Adopted modificatins to USR 4/5/6 time interpretation per suggestion from tsteven4.

* Adopted modificatins to USR 4/5/6 time interpretation per suggestion from tsteven4.

* Changed file header parsing back to int16 for USR format as found sample file with a stream version other than 0.  Assume this
is true for ALL USR formats until determine otherwise.

* Updated reference files after corrections to altitude and time conversion.

* Check for both "nan" and "-nan" to address compiler differences.

* Use -options instead of --options to avoid osx OS issues.

* Set altitude to "unknown_alt" when creating Waypoints from Event ICONs.
Updated reference files to encorporate "unknown_alt" change.
Corrected mis-spelling of "green"

* Change cut options from --compliment to legacy explict field list for OSX OS support.

* Applied codex patch from tsteven4 - thank you.

* Corrected interpretation of elevation in USR2/3.

* Corrected reference files and test script with changes to elevation interpretation.

* More conversions to QString conventions.
Minor correction to trail attribute parsing.

* Editorial cleanup.

* USRv6 uses trail data stream version 5 which employs int32 rather than char for attributes.

* Use invalid ICON value as method to detect HOOK2 generated USR 2/3 files.

* Cleanup time conversions.

Use QList instead of QueueList to back RouteList.

This eliminates the usage of the legacy queue code for route lists.

Merge remote-tracking branch 'origin/master' into routelist

bug fixes for xcsv reader. (#295)

For some lines data must be accumulated from several fields
in arbitrary field order.
Previously this was done with global variables.  Now a variable to
hold the accumulated data for a line is constructed/destructed for
each line.

This eliminates the possiblity of unintended communication between
lines.

This also eliminated some unintended communication between the reader
and the writer through global variables that they previously shared
(csv_track, csv_route).

An ordering sensitivty to TRACK_NEW and TRACK_NAME is removed.

For the reader, conditional allocation of route_heads is removed
from xcsv_parse_val, and centralized in xcsv_data_read.
This eliminates some undetected
memory leaks which hid as "still reachable" due to the pointer
being saved in a global variable (csv_track, csv_route).

Undesirable reading of the route_head->Q variable is
eliminated improving encapsulation.  I beleive the intent was
never to decide if the csv_track->Q was empty, i.e. it hadn't been
added to the global track_head yet.  I beleive what was desired was
to decide if csv_track had any waypoints. In any event the
centralization of route_head allocation and head addition makes new
track handling much more straight forward and robust.

The xcsv test case is enhanced to increase coverage.

Introduce a RouteList class.

This is backed by a QueueList class, which
adds iterators for queues.  It is anticipated that
this will be replaced by usage of a more standard
container.

skytraq: some minor cleanup I had lying around...

skytraq: rename option gps_utc_offset to gps-utc-offset, to be consistent with other options

skytraq: add doc for new option gps-week-rollover

skytraq: prepare for upcoming gps week-rollover